-
Notifications
You must be signed in to change notification settings - Fork 202
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add cuda_cc_space_sep
template that does not have periods
#4583
Add cuda_cc_space_sep
template that does not have periods
#4583
Conversation
…' if cuda compute capabilities is 8.0,9.0
…_capabilities, and not the local variable
…s build option, instead of the local var
cuda_cc_space_sep
template that does not have periods
@@ -94,9 +94,11 @@ | |||
"as specify by the --sysroot configuration option"), | |||
('mpi_cmd_prefix', "Prefix command for running MPI programs (with default number of ranks)"), | |||
('cuda_compute_capabilities', "Comma-separated list of CUDA compute capabilities, as specified via " | |||
"--cuda-compute-capabilities configuration option or via cuda_compute_capabilities easyconfig parameter"), | |||
"--cuda-compute-capabilities configuration option or via cuda_cc easyconfig parameter"), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@casparvl Maybe I'm missing something, but there's no cuda_cc
easyconfig parameter in EasyBuild 4.x (nor in the 5.0.x
branch), so this change is incorrect?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
being fixed in #4589
E.g. '80 90' if cuda compute capabilities is '8.0,9.0'
Can be used in e.g. Cuda-Samples as value to
SMS
argument, see the readme at https://github.com/NVIDIA/cuda-samples